Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Evan Thomas Chitter Challenge #2187

Open
wants to merge 56 commits into
base: main
Choose a base branch
from
Open

Evan Thomas Chitter Challenge #2187

wants to merge 56 commits into from

Conversation

ev-th
Copy link

@ev-th ev-th commented May 11, 2023

No description provided.

ev-th added 30 commits May 5, 2023 15:23
@ev-th
Set up rspec folder structure and sinatra config
3cf4eaa
@ev-th
Write sql seeds files
686a27c
@ev-th
Set up empty database files
7ae5ca5
@ev-th
Write UserRepository#find
e04cb8d
@ev-th
Write UserRepository#all and UserRepository#create
b41415d
@ev-th
Write PeepRepository#all
2a3a279
@ev-th
Write PeepRepository#find and PeepRepository#create
880717b
@ev-th
Rewrite DatabaseConnection to allow for deployment with repote database
29888d6
@ev-th
Set up app.rb and move reset tables methods into spec_helper
38ba99b
@ev-th
Write the skeleton for http route tests
d00069c
@ev-th
Fix bug with getting timestamp from the database
e987bf8
@ev-th
Write GET / route and initial homepage html
469293e
@ev-th
Write GET /sign-up route and sign up page html
e3ae33e
@ev-th
Write GET /new-peep route and html form:
ae84c4d
@ev-th
Write POST /sign-up route with success and html
3065380
@ev-th
Write POST /new-peep route for valid params and html success page
7bb239c
@ev-th
Remove time added from new peep form due to dynamic generation
c97f8a3
@ev-th
Update GET / to return peeps in reverse chronological order
e022178
@ev-th
Fix rubocop offenses
10c1494
@ev-th
sanitise new-user input to check for empty strings
d38fe32
@ev-th
Sanitise user input for invalid characters
b651d7e
@ev-th
Validate input for POST /new-peep
6bc43ca
@ev-th
Write UserRepository#username_exists? and #email_exists
a8cbb33
@ev-th
Update POST /sign-up to only allow new usernames and emails to sign up
ddbc07a
@ev-th
Use bcrypt to encyrpt passwords
dd44160
@ev-th
Make GET /login and html page
16ff5c3
@ev-th
Write UserRepository#find_by_email
b3bf420
@ev-th
Start writing the tests and implementation for logging in the user
d6ae431
@ev-th
Write UserRepository#correct_password
547554e
@ev-th
Implement user login
9938503
ev-th added 26 commits May 9, 2023 16:23
@ev-th
UserRepository#find_by_email returns nil if not in db
c93e935
@ev-th
Create login failure page and do not login with wrong password
a51ce00
@ev-th
Only show login and sign up links when not logged in
c872fc8
@ev-th
implement logout
ce770e9 
:
@ev-th
Use login session to assign new peeps to the logged in user
71088c1
@ev-th
Hompage displays name with peeps
02364fc
@ev-th
Add .env to .gitignore
2797a84
@ev-th
Set up dotenv and EmailSender class
67d5a0c
@ev-th
Implement EmailSender
162d3c8
@ev-th
Begin writing EmailSender test
937354e
@ev-th
Write UserRepository#find_by_username
ace0311
@ev-th
Email user when tagged in peep
93cd1d9
@ev-th
Add some css styling
ea318d8
@ev-th
Update a test to pass with changed html tag
7d502dd
@ev-th
Update peep display with new background image
5bf835f
@ev-th
Change the bcrypt hashing cost
a071798
@ev-th
Setup for deploy
12328a8
@ev-th
Remove quotes from css url functions
19170b8
@ev-th
Write Peep#formatted_time
48846b1
@ev-th
Update styling to make small font more readable. Fix html text on new…
de8f0be 
… peep page
@ev-th
Change date formatting of peep date and time on the homepage
1ab159d
@ev-th
Restyle the peeps: Translucent background resized fonts, reordered info
1e255ea
@ev-th
Refactor EmailSender
1e43f0f
@ev-th
Refactor for style guide
adbc99f
@ev-th
Extract logging in a user in tests to a method
4d20e9f
@ev-th
Write README.md
ce69d21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ev-th